Skip to content

🔨 remove Helicone proxy integration#69

Merged
Miyamura80 merged 1 commit into
mainfrom
remove/helicone
Jan 26, 2026
Merged

🔨 remove Helicone proxy integration#69
Miyamura80 merged 1 commit into
mainfrom
remove/helicone

Conversation

@Miyamura80

Copy link
Copy Markdown
Owner

Summary

  • Remove unused api_base() method from common/global_config.py that returned Helicone proxy URLs (hconeai.com, helicone.ai)
  • The method was never called anywhere in the codebase

Test plan

  • make ruff passes
  • make test_fast passes (4 tests, 23% coverage)

🤖 Generated with Claude Code

Remove unused api_base() method that returned Helicone proxy URLs.
The method was never called anywhere in the codebase.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@greptile-apps

greptile-apps Bot commented Jan 26, 2026

Copy link
Copy Markdown
Contributor

Greptile Overview

Greptile Summary

This PR removes the unused api_base() method from common/global_config.py that was designed to return Helicone proxy URLs for various LLM providers (OpenAI, Groq, Perplexity, Gemini).

  • The method was never called anywhere in the codebase (verified via grep search)
  • No references to Helicone or hconeai exist in the project
  • No tests referenced this method
  • No documentation mentioned this functionality
  • The removal simplifies the Config class by removing dead code
  • The related llm_api_key() method remains intact and functional

This is a clean removal with no breaking changes.

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • The change removes genuinely unused code with no dependencies. Comprehensive searches confirm the method was never called, no tests break, and no documentation needs updating. The PR author verified tests pass (make test_fast and make ruff).
  • No files require special attention

Important Files Changed

Filename Overview
common/global_config.py Removed unused api_base() method that returned Helicone proxy URLs; no references found in codebase

@Miyamura80 Miyamura80 merged commit 5932f80 into main Jan 26, 2026
8 checks passed
@github-actions github-actions Bot deleted the remove/helicone branch January 26, 2026 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant